Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

B2CA-1838: Implement swap #82

Merged
merged 28 commits into from
Dec 12, 2024
Merged

Conversation

cedelavergne-ledger
Copy link

@cedelavergne-ledger cedelavergne-ledger commented Oct 18, 2024

Implementation of the SWAP in app-cardano. This also requires:

  • Porting some legacy TypeScript tests on ragger framework
  • Port the App on Standard App (Makefile + Code fix)
  • Implement the swap itself

Note: the fail test on the CI workflow is only due to the version check. Those tests are located in a separate repository (https://github.com/vacuumlabs/ledgerjs-cardano-shelley), where I do not have the permission to create a PR to update it

Checklist

  • App update process has been followed
  • Target branch is develop
  • Application version has been bumped

@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1838_implement-swap branch 25 times, most recently from ca7b314 to de9eaaf Compare October 25, 2024 15:09
@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1838_implement-swap branch 4 times, most recently from d69e21e to fc4b618 Compare October 30, 2024 17:13
@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1838_implement-swap branch from 6b3be4e to 7a2f130 Compare December 11, 2024 17:37
bboilot-ledger
bboilot-ledger previously approved these changes Dec 12, 2024
@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1838_implement-swap branch from 7bd146d to 861f5bc Compare December 12, 2024 10:43
@cedelavergne-ledger cedelavergne-ledger merged commit 908f4df into develop Dec 12, 2024
3 checks passed
@cedelavergne-ledger cedelavergne-ledger deleted the cev/B2CA-1838_implement-swap branch December 12, 2024 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants